[Feat] Add option to hide MCP descriptions - #1489
Conversation
Review statusThis PR was opened by an automated account. A human maintainer must verify the change intent, provenance, and validation before merging. Current step: Resolve the merge conflicts. The review sequence resumes after the branch is mergeable. Review-state labels are managed by this workflow; do not edit them manually. |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What changed
Show MCP descriptionssetting to MCP settings, persisted through the existing global settings mechanism and defaulted on to preserve current behavior.Why this change was made
Verbose descriptions supplied by MCP servers can add substantial visual noise. Users can now reduce that clutter without disabling MCP capabilities or hiding functional metadata.
Impact
Existing users continue to see descriptions unless they turn the setting off. Saved changes update MCP settings immediately through cached state and chat through the refreshed extension state after Save.
Browser screenshots are not included because Zoo-Code's changed surface runs only inside the VS Code F5 extension host, while this task's browser target exposes only the documentation site.
Related PRs
Validation
pnpm exec vitest run src/__tests__/global-settings.test.tsfrompackages/types: 1 file, 6 tests passed.pnpm exec vitest run src/components/mcp/__tests__/McpToolRow.spec.tsx src/components/mcp/__tests__/McpResourceRow.spec.tsx src/components/mcp/__tests__/McpView.spec.tsx src/components/settings/__tests__/SettingsView.unsaved-changes.spec.tsxfromwebview-ui: 4 files, 35 tests passed.pnpm exec vitest run core/webview/__tests__/webviewMessageHandler.spec.ts core/webview/__tests__/ClineProvider.spec.tsfromsrc: 2 files, 239 tests passed.node scripts/find-missing-translations.js: all translations complete.pnpm check-types: 11 tasks passed.pnpm lintvia the commit hook: 11 tasks passed.pnpm test: 464 test files and 7,951 tests passed; 4 files and 39 tests skipped.git diff --check origin/main...HEAD: passed.